Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

go mod tidy for lotus-soup testplans #6124

Merged
merged 1 commit into from
Apr 29, 2021
Merged

go mod tidy for lotus-soup testplans #6124

merged 1 commit into from
Apr 29, 2021

Conversation

nonsense
Copy link
Member

Fixup for #6122

Since CircleCI runs only go get lotus@master, but not go mod tidy, sometimes we have to clean the go.mod of old libraries in order to avoid compile errors:

Step 15/22 : RUN cd ${PLAN_DIR}     && go env -w GOPROXY="${GO_PROXY}"     && CGO_ENABLED=${CgoEnabled} GOOS=linux GOARCH=amd64 go build -o ${PLAN_DIR}/testplan.bin ${BUILD_TAGS} ${TESTPLAN_EXEC_PKG}
 ---> Running in d0cd81ce0cd5
�[91m/go/pkg/mod/github.com/filecoin-project/[email protected]/api/api_full.go:14:2: missing go.sum entry for module providing package github.com/filecoin-project/go-data-transfer (imported by github.com/filecoin-project/lotus/api); to add:
	go get github.com/filecoin-project/lotus/[email protected]
/go/pkg/mod/github.com/filecoin-project/[email protected]/node/modules/client.go:13:2: missing go.sum entry for module providing package github.com/filecoin-project/go-data-transfer/channelmonitor (imported by github.com/filecoin-project/lotus/node/modules); to add:
	go get github.com/filecoin-project/lotus/node/[email protected]
/go/pkg/mod/github.com/filecoin-project/[email protected]/node/modules/client.go:14:2: missing go.sum entry for module providing package github.com/filecoin-project/go-data-transfer/impl (imported by github.com/filecoin-project/lotus/node/modules); to add:
	go get github.com/filecoin-project/lotus/node/[email protected]
...

@nonsense nonsense requested a review from magik6k April 28, 2021 12:37
@nonsense nonsense requested a review from arajasek as a code owner April 28, 2021 12:37
@nonsense nonsense mentioned this pull request Apr 29, 2021
Merged
@magik6k magik6k merged commit 492b026 into master Apr 29, 2021
@magik6k magik6k deleted the testplans-go-mod branch April 29, 2021 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@magik6k magik6k magik6k approved these changes

@arajasek arajasek Awaiting requested review from arajasek

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nonsense @magik6k